The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		May 20, 2026 7:04pm

PR Summary

Medium Risk
Medium risk because it changes optimistic run-state accounting and dispatch overlay seeding in React Query, which can affect live run counters and cancellation UI if mismatched with SSE/server state. Server-side changes are mostly lazy-loading/perf optimizations but touch trigger firing and job enqueue options.

Overview
Improves table workflow run responsiveness and UI correctness by reducing cold-start costs, fixing live running counters, and smoothing cell reveal animations.

On the client, useRunColumn now tracks how many cells are optimistically stamped pending, bumps runningCellCount/runningByRowId in useTableRunState to match, rolls back on error, and seeds the new dispatch into the active-dispatch overlay from the mutation response (avoiding a refetch that would zero out the optimistic count).

On the server, fireTableTrigger lazily imports webhook utilities/processor, and buildEnqueueItems only imports/provides the inline runner when not using trigger.dev. In the grid, the typewriter effect switches from per-cell setInterval to elapsed-time requestAnimationFrame updates to batch renders across many cells.

^{Reviewed by Cursor Bugbot for commit 745a5a3. Bugbot is set up for automated code reviews on this repo. Configure here.}

Greptile Summary

Three targeted bug fixes following the chunked-dispatcher merge: a dispatcher cold-start reduction via lazy imports, an optimistic live-counter fix for the "X running" badge, and a requestAnimationFrame-based typewriter to batch concurrent cell reveals into a single paint per frame.

• trigger.ts / workflow-columns.ts: fetchActiveWebhooks and processPolledWebhookEvent are now lazy-imported inside fireTableTrigger, and buildEnqueueItems skips the executor import entirely on trigger.dev — both changes prevent the workflow-executor stack from loading in the dispatcher container at startup.
• tables.ts: onMutate in useRunColumn now bumps runningCellCount/runningByRowId optimistically (with rollback on error), and onSuccess seeds the dispatch directly from the response rather than calling invalidateQueries (which would have reset the counter to the server's still-zero value).
• cell-render.tsx: Typewriter switches from setInterval to an elapsed-time requestAnimationFrame loop so all concurrent cell reveals share one render+paint cycle per frame.

Confidence Score: 3/5

Safe to merge for the cold-start and typewriter fixes; the live-counter change has a gap where the counter can become permanently stuck if the server returns success without a dispatchId.

The onSuccess handler replaces invalidateQueries with a targeted cache update, but returns early without any counter cleanup when dispatchId is undefined. In that scenario the optimistic runningCellCount bump from onMutate has no rollback path — onError won't fire (the mutation succeeded) and no SSE will arrive for a dispatch that was never created, leaving the badge stuck. The lazy-import and rAF changes are straightforward and carry no meaningful risk.

apps/sim/hooks/queries/tables.ts — specifically the onSuccess handler in useRunColumn

Important Files Changed

Sequence Diagram

sequenceDiagram
    participant UI as UI (useRunColumn)
    participant QC as QueryClient Cache
    participant API as Run Column API
    participant SSE as SSE Stream

    UI->>QC: onMutate: stamp cells pending, bump runningCellCount optimistically
    UI->>API: POST /run-column
    API-->>UI: "200 { dispatchId }"
    alt dispatchId present
        UI->>QC: onSuccess: upsert ActiveDispatch into cache (counter preserved)
    else dispatchId absent
        UI->>UI: return early — counter NOT rolled back
    end
    SSE-->>QC: cell events update counter as dispatcher processes rows
    Note over UI,SSE: onError rolls back both row snapshots and counter bump

_{Reviews (1): Last reviewed commit: "fix(table): drive cell typewriter with r..." | Re-trigger Greptile}